Skip to content

[pentest/gdb] Cleanup gdb tests and add the rom_ext_imm test - #31270

Merged
siemen11 merged 1 commit into
lowRISC:earlgrey_1.0.0from
siemen11:gdb_test_cleanup_earlgrey
Sep 10, 2026
Merged

[pentest/gdb] Cleanup gdb tests and add the rom_ext_imm test#31270
siemen11 merged 1 commit into
lowRISC:earlgrey_1.0.0from
siemen11:gdb_test_cleanup_earlgrey

Conversation

@siemen11

@siemen11 siemen11 commented Sep 9, 2026

Copy link
Copy Markdown
Contributor

The GDB fault simulations can be made faster and more accurate (no missed PCs) using better handling of openocd:

  • Breakpoint misfire recovery: if jtag/openocd delays happen, do not abort the campaign but retry
  • Connection check: check gdb health by pinging it and extend openocd restart delays
  • UART clean: empty UART at every test to ensure no stale data is read out
  • Switch to hardware breakpoints for observation points

Add tags to run a specific test in a larger python unittest. Fix the rom_ext_imm double otp read issue that was bricking gdb.

Also add the rom_ext_imm test from master.

@siemen11
siemen11 force-pushed the gdb_test_cleanup_earlgrey branch from 8f7f100 to 0044298 Compare September 9, 2026 13:50
@siemen11 siemen11 changed the title [pentest/gdb] Cleanup gdb tests [pentest/gdb] Cleanup gdb tests and add the rom_ext_imm test Sep 9, 2026
@siemen11
siemen11 force-pushed the gdb_test_cleanup_earlgrey branch from 0044298 to 134b8aa Compare September 9, 2026 15:07
@siemen11
siemen11 requested a review from cfrantz as a code owner September 9, 2026 15:07
@siemen11
siemen11 force-pushed the gdb_test_cleanup_earlgrey branch from 134b8aa to 371bcfe Compare September 9, 2026 19:36
@siemen11
siemen11 requested review from nasahlpa and removed request for cfrantz September 9, 2026 19:37
@siemen11
siemen11 force-pushed the gdb_test_cleanup_earlgrey branch from 371bcfe to 1567392 Compare September 9, 2026 19:41
The GDB fault simulations can be made faster and more accurate (no
missed PCs) using better handling of openocd:
- Breakpoint misfire recovery: if jtag/openocd delays happen, do not
  abort the campaign but retry
- Connection check: check gdb health by pinging it and extend openocd
  restart delays
- UART clean: empty UART at every test to ensure no stale data is read
  out
- Switch to hardware breakpoints for observation points

Add tags to run a specific test in a larger python unittest.
Fix the rom_ext_imm double otp read issue that was bricking gdb.

Add the rom_ext_imm test from master.

Signed-off-by: Siemen Dhooghe <sdhooghe@google.com>
@siemen11
siemen11 force-pushed the gdb_test_cleanup_earlgrey branch from 1567392 to 381c982 Compare September 10, 2026 07:50
@siemen11 siemen11 added the CI:Rerun Rerun failed CI jobs label Sep 10, 2026
@github-actions github-actions Bot removed the CI:Rerun Rerun failed CI jobs label Sep 10, 2026
@siemen11
siemen11 merged commit e06cbca into lowRISC:earlgrey_1.0.0 Sep 10, 2026
69 of 70 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants